Skip to content

Guinevere's FarMar #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Guinevere's FarMar #45

wants to merge 24 commits into from

Conversation

@guineveresaenger
Copy link

No description provided.

@guineveresaenger
File setup
e1ce9a0
@guineveresaenger
Added all and find class methods to all classes, including tests
e27a57f
@guineveresaenger
Updated .gitignore to include extraneous coverage folder. Added vendo…
52dca47 
…r method to Market class with test.
@guineveresaenger
Fixed the test for vendors method
94539d6
@guineveresaenger
Fixed the vendors method to actually return Vendor objects in an array
a59deda
@guineveresaenger
Updated vendor and vendor specs to have a market method
7c57a62
@guineveresaenger
Added self.by_market method to vendor; updated tests and Market class
a90240a
@guineveresaenger
Added self.by_vendor method to Product
ac38f98
@guineveresaenger
Added products method to Vendor, and self.by_vendor to Product.
6697f0f
@guineveresaenger
Fixed initialize test in product specs; added vendor method to Product
80ee00c
@guineveresaenger
Added test for vendor method for Sale class
8d62d74
@guineveresaenger
Added vendor method to Sales class; added Sales method to Vendor class.
adaadf4
@guineveresaenger
Cleaned up some of the tests
a82016a
@guineveresaenger
Updated sales method in vendor to use Sales.all method
f8e8782
@guineveresaenger
contains actual updated sales method
e01dc62
@guineveresaenger
Added sales method to Product class
594d073
@guineveresaenger
Added number_of_sales method ot Product
f9520da
@guineveresaenger
Updated self.by_vendor to use self.all method
88f65ae
@guineveresaenger
Updated self.by_market to use self.all method
22c19c8
@guineveresaenger
Added revenue method in Vendor
6297a12
@guineveresaenger
Updated revenue method with &: in call to map
61db1e9
@guineveresaenger
Updated the self.find method blocks to have curly braces
0194492
@guineveresaenger
Added self.between method for Sale class, with specs. Project is now …
c20651c 
…fully functional according to Primary Requirements.
@guineveresaenger
Copy link
Author

Meets all Primary Requirements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@guineveresaenger